Skip to content

feat: Implement Rokt selectPlacements#34

Merged
jamesnrokt merged 22 commits into
mainfrom
feat/SQDSDKS-7523-Implement-Rokt
Aug 4, 2025
Merged

feat: Implement Rokt selectPlacements#34
jamesnrokt merged 22 commits into
mainfrom
feat/SQDSDKS-7523-Implement-Rokt

Conversation

@jamesnrokt

Copy link
Copy Markdown
Contributor

Instructions

  1. PR target branch should be against development
  2. PR title name should follow this format: https://github.com/mParticle/mparticle-workflows/blob/main/.github/workflows/pr-title-check.yml
  3. PR branch prefix should follow this format: https://github.com/mParticle/mparticle-workflows/blob/main/.github/workflows/pr-branch-check-name.yml

Summary

  • Add support for Rokt selectPlacements

Testing Plan

  • Was this tested locally? If not, explain why.
  • Tested on Android and iOS using the example application
Android iOS

Reference Issue (For mParticle employees only. Ignore if you are an outside contributor)

Copilot AI review requested due to automatic review settings July 24, 2025 05:48
@jamesnrokt jamesnrokt requested a review from a team as a code owner July 24, 2025 05:48

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR implements support for Rokt's selectPlacements functionality in the MParticle Cordova plugin. The implementation provides a cross-platform interface for configuring and executing Rokt placement selection with support for color modes, caching, and display options.

Key Changes

  • Add selectPlacements JavaScript API with configuration options for color mode, caching, and edge-to-edge display
  • Implement native iOS and Android selectPlacements methods that bridge to the respective Rokt SDKs
  • Update SDK dependencies to support Rokt integration (mParticle SDK v5.71.0 for Android, v8.35.0 for iOS)

Reviewed Changes

Copilot reviewed 19 out of 22 changed files in this pull request and generated 4 comments.

Show a summary per file
File Description
plugin/www/mparticle.js Adds selectPlacements function and RoktColorMode enum with config merging logic
plugin/src/ios/CDVMParticle.m Implements iOS selectPlacements bridge and fixes user identities conversion
plugin/src/android/.../MParticleCordovaPlugin.java Implements Android selectPlacements bridge with Rokt SDK integration
plugin/plugin.xml Updates SDK versions and iOS podspec configuration
example/ files Comprehensive example application updates with testing interface and build improvements
Files not reviewed (2)
  • example/package-lock.json: Language not supported
  • plugin/package-lock.json: Language not supported

Comment thread plugin/src/ios/CDVMParticle.m
Comment thread plugin/www/mparticle.js Outdated
Comment thread plugin/src/ios/CDVMParticle.m
rmi22186
rmi22186 previously approved these changes Jul 24, 2025

@rmi22186 rmi22186 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

the logic looks good to me.

@BrandonStalnaker BrandonStalnaker left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@jamesnrokt jamesnrokt merged commit 2c16a13 into main Aug 4, 2025
11 checks passed
@jamesnrokt jamesnrokt deleted the feat/SQDSDKS-7523-Implement-Rokt branch August 4, 2025 05:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants